The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Jun 15, 2026 1:59am

@greptile

@cursor review

PR Summary

Low Risk
UI-only keyboard shortcut and tooltip wiring; no auth, data, or API changes. Mod+B may overlap with browser bold in some contexts, but the command is disabled in editable fields.

Overview
Adds a workspace-wide Mod+B (collapse-sidebar) global command that toggles sidebar collapse via the existing sidebar store, with allowInEditable: false so it does not fire while typing in inputs.

Introduces formatCommandShortcut in the command registry so tooltip labels stay aligned with registered bindings (macOS symbols vs Ctrl+ labels elsewhere). The collapse control and collapsed Expand sidebar affordance show that shortcut through Tooltip.Shortcut.

SidebarTooltip gains an optional shortcut prop for the same pattern on the collapse button.

^{Reviewed by Cursor Bugbot for commit 97d8c98. Configure here.}

Greptile Summary

This PR adds a ⌘B / Ctrl+B keyboard shortcut to toggle the workspace sidebar, wiring it through the central command registry (collapse-sidebar, Mod+B, allowInEditable: false) and surfacing it in both the collapse button tooltip and the expand affordance in WorkspaceHeader. A new formatCommandShortcut helper derives display strings from COMMAND_DEFINITIONS, eliminating the risk of tooltips drifting from the registered binding.

• Registry entry (commands-utils.ts): adds 'collapse-sidebar' to CommandId, declares Mod+B with allowInEditable: false (matching run-workflow and clear-terminal-console to preserve native bold in editable fields), and adds formatCommandShortcut for platform-aware tooltip labels.
• Sidebar (sidebar.tsx): registers the handler via useRegisterGlobalCommands, extends SidebarTooltip with an optional shortcut prop, and passes the formatted shortcut to both the collapse button and WorkspaceHeader.
• WorkspaceHeader (workspace-header.tsx): wraps the collapsed-state expand button in a Tooltip.Root/Trigger/Content tree so the ⌘B hint is discoverable from both open and collapsed states.

Confidence Score: 5/5

Safe to merge — the shortcut is correctly scoped to non-editable contexts, tooltips derive from a single source of truth, and both the collapse and expand affordances surface the hint.

The three previously flagged concerns are all resolved in this revision. isMacPlatform() is SSR-safe, formatCommandShortcut is pure and correct for multi-token shortcut strings, and the Tooltip wrapper in WorkspaceHeader degrades cleanly when expandShortcut is omitted. No new logic errors or regressions introduced.

No files require special attention.

Important Files Changed

_{Reviews (5): Last reviewed commit: "improvement(sidebar): address review — s..." | Re-trigger Greptile}

Greptile Summary

This PR registers ⌘B / Ctrl+B as a global shortcut to collapse/expand the workspace sidebar, wires it through the central COMMAND_DEFINITIONS registry, and surfaces a platform-aware key hint in the collapse button's tooltip via Tooltip.Shortcut.

• collapse-sidebar is added to COMMAND_DEFINITIONS with shortcut: 'Mod+B' and allowInEditable: true; the sidebar registers a handler that calls toggleCollapsed().
• SidebarTooltip gains an optional shortcut prop that switches from a plain <p> to a Tooltip.Shortcut when provided; the collapse button passes isMac ? '⌘B' : 'Ctrl+B'.
• The shortcut hint is only surfaced on the collapse button (sidebar open state); the expand path in WorkspaceHeader does not carry the hint, leaving a discoverability gap for users who first see the collapsed sidebar.

Confidence Score: 4/5

Safe to merge; the changes are small and self-contained, with no effect on data or auth. The main practical risk is Mod+B being intercepted in text inputs/contenteditable elements, which could break bold formatting if rich-text editors exist in the workspace.

The implementation is clean and consistent with how other global shortcuts are wired. The allowInEditable: true on Mod+B is the one decision that could cause unexpected behavior: the capture-phase listener calls preventDefault() unconditionally, so any future (or existing) rich-text editor would silently lose its bold shortcut. The tooltip string is also hardcoded rather than derived from the registry, which is a minor drift risk. Neither concern is blocking, but the allowInEditable choice is worth deliberate confirmation from the team.

commands-utils.ts — specifically the allowInEditable: true decision for collapse-sidebar deserves a second look before merging.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant User
    participant GlobalCommandsProvider
    participant Sidebar

    User->>GlobalCommandsProvider: keydown ⌘B / Ctrl+B (capture phase)
    GlobalCommandsProvider->>GlobalCommandsProvider: matchesShortcut("collapse-sidebar")
    GlobalCommandsProvider->>GlobalCommandsProvider: e.preventDefault() + e.stopPropagation()
    GlobalCommandsProvider->>Sidebar: handler() → toggleCollapsed()
    Sidebar->>Sidebar: isCollapsed state flipped

    Note over User,Sidebar: Tooltip path (sidebar expanded)
    User->>Sidebar: hover collapse button
    Sidebar->>User: SidebarTooltip shows "Collapse sidebar ⌘B"

_{Reviews (2): Last reviewed commit: "feat(sidebar): add ⌘B shortcut to collap..." | Re-trigger Greptile}

@greptile

@cursor review

@greptile review